Skip to content

fix sort order to be consistent across OSes;...#2176

Merged
kim-tsao merged 2 commits into
mainfrom
sort-preinstalled
May 8, 2026
Merged

fix sort order to be consistent across OSes;...#2176
kim-tsao merged 2 commits into
mainfrom
sort-preinstalled

Conversation

@nickboldt

Copy link
Copy Markdown
Member

What does this PR do?

fix sort order to be consistent across OSes; sort the preinstalled list too

Signed-off-by: Nick Boldt nboldt@redhat.com

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

N/A (or see commit message above for issue number)

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

  • Code produced is complete
  • Code builds without errors
  • Tests are covering the bugfix
  • Relevant user documentation updated
  • Relevant contributing documentation updated

Reviewers

Reviewers, please comment how you tested the PR when approving it.

nickboldt added 2 commits May 8, 2026 12:59
…st too

Signed-off-by: Nick Boldt <nboldt@redhat.com>
…st too

Signed-off-by: Nick Boldt <nboldt@redhat.com>
@sonarqubecloud

sonarqubecloud Bot commented May 8, 2026

Copy link
Copy Markdown

@rhdh-bot

rhdh-bot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Shellcheck Analysis Results

Changed shell scripts in this PR: 1

Scripts analyzed:

  • modules/extend_dynamic-plugins-reference/rhdh-supported-plugins.sh

Check the Files changed tab for detailed shellcheck suggestions.

All findings are reported as warnings and won't block the PR.


Automated shellcheck analysis • See shellcheck.net for details

@rhdh-bot

rhdh-bot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

PR Build Results

Build passed -- 34/34 titles | 75s
Preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-2176/


Content Quality Assessment

CQA Report

  • CQA-00a: Orphaned modules
  • CQA-00b: Directory structure
  • CQA-01: Vale AsciiDoc DITA compliance
  • CQA-02: Verify assembly structure
  • CQA-03: Verify content type metadata
  • CQA-04: Verify module templates
  • CQA-05: Verify required modular elements
  • CQA-06: Verify assemblies follow official template (one user story)
  • CQA-07: Verify TOC depth (max 3 levels)
  • CQA-08: Verify short description content quality
  • CQA-09: Verify short description format
  • CQA-10: Verify titles are brief, complete, and descriptive
  • CQA-11: Verify procedure prerequisites
  • CQA-12: Verify grammar and style (Vale)
  • CQA-13: Verify content matches declared type
  • CQA-14: Verify no broken links
  • CQA-15: Check redirects
  • CQA-16: Verify official product names
  • CQA-17: Verify legal disclaimers for preview features

Summary

Checks: 19 total, 19 pass, 0 fail

19 checks: 19 pass, 0 fail

Run node build/scripts/cqa/index.js --all --fix locally to review and auto-fix issues.


Updated 2026-05-08 16:04:53 UTC

@kim-tsao kim-tsao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 8, 2026
@kim-tsao kim-tsao merged commit da42a1e into main May 8, 2026
10 checks passed
@kim-tsao kim-tsao deleted the sort-preinstalled branch May 8, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants